Skip to content

Conversation

@PiJoules
Copy link
Contributor

No description provided.

@PiJoules PiJoules requested review from lntue and petrhosek February 27, 2025 22:42
@llvmbot llvmbot added the libc label Feb 27, 2025
@llvmbot
Copy link
Member

llvmbot commented Feb 27, 2025

@llvm/pr-subscribers-libc

Author: None (PiJoules)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/129138.diff

1 Files Affected:

  • (modified) libc/docs/headers/index.rst (+1)
diff --git a/libc/docs/headers/index.rst b/libc/docs/headers/index.rst
index 3dc30ef90a8e4..e569a22d53a93 100644
--- a/libc/docs/headers/index.rst
+++ b/libc/docs/headers/index.rst
@@ -23,6 +23,7 @@ Implementation Status
    setjmp
    signal
    stdbit
+   stdfix <math/stdfix.rst>
    stdio
    stdlib
    string

setjmp
signal
stdbit
stdfix <math/stdfix.rst>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd move stdfix.rst to top-level to match other headers.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like this?

@PiJoules PiJoules merged commit 1a09ada into llvm:main Mar 6, 2025
12 of 16 checks passed
@PiJoules PiJoules deleted the stdfix-link branch March 6, 2025 18:31
boomanaiden154 added a commit to boomanaiden154/llvm-project that referenced this pull request Mar 6, 2025
The docs build action was failing with libc due to checks.rst not
existing in the expected path. This patch adjusts the path to the actual
path which seems to make everything happy. It seems like this did not
show up before as stdfix.rst was not included in a place that actually
caused it to get picked up by sphinx.
boomanaiden154 added a commit that referenced this pull request Mar 6, 2025
The docs build action was failing with libc due to checks.rst not
existing in the expected path. This patch adjusts the path to the actual
path which seems to make everything happy. It seems like this did not
show up before as stdfix.rst was not included in a place that actually
caused it to get picked up by sphinx.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants